Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds Vue Dashboard for Realtime TES Tracking #27

Merged
merged 1 commit into from
Sep 17, 2024
Merged

Adds Vue Dashboard for Realtime TES Tracking #27

merged 1 commit into from
Sep 17, 2024

Conversation

michaelgattas
Copy link
Contributor

• ✨ Adds Vue Dashboard (using Axios, Vuetify, and a Primevue component) for realtime tracking of TES Tasks
• 🙈 Adds node_modules/ directory to the .gitignore
• 💩 Includes .env files for the sake of transferring project over (please add to .gitignore afterward)

Copy link

gitguardian bot commented Sep 10, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.


/** Accesses Axios headers and adds necessary authorization header **/
setHeader() {
axios.defaults.headers.common['Authorization'] = `Basic dGVzLXJ1bm5lcjo4L0VhWVxQUXo1MH4=`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@michaelgattas would you be able to remove this piece?

Note for future readers, the K8s server we used for demoing has been deleted, so the leaking of this key is inconsequential—we should just remove the setting of the header for any future users of the dashboard (since that's not relevant).

Copy link
Contributor Author

@michaelgattas michaelgattas Sep 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@claymcleod Do you want me to go ahead and remove the Token Service and all header/token-based code? This was all still left-over from my boilerplate project

@michaelgattas michaelgattas merged commit a89eed7 into main Sep 17, 2024
6 checks passed
@michaelgattas michaelgattas deleted the webapp branch September 17, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants